-
Notifications
You must be signed in to change notification settings - Fork 181
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Exposing environment settings. #2921
base: main
Are you sure you want to change the base?
Conversation
What does GetEnvironmentSetting actually return? |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Why do you even need this?
Do we know the exact values that EnvironmentSettings can contain?
From a security pov, I don't like this.
src/System Application/App/Environment Information/src/EnvironmentInformationImpl.Codeunit.al
Outdated
Show resolved
Hide resolved
To control the visibility of the e-document flows that will not be visible in v26.0 when released, but that we still want feedback from private previews organized by the PM. The approach that was agreed on was to change a setting from Cloud Manager. I added OnPrem, and the caller module filtering calls from "Microsoft" published apps. Alternatively, we need an exception to call dotnet directly from the edocs app - that could be better than adding this, should I do that and abandon this PR? |
…te/diegojoshuam/envsettings
paused until it goes through security review |
Exposing the EnvironmentSettings for Microsoft published callers.
AB#565525